Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Original onnx #6673

Merged
merged 7 commits into from
Aug 4, 2021
Merged

Original onnx #6673

merged 7 commits into from
Aug 4, 2021

Conversation

ilyachur
Copy link
Contributor

@ilyachur ilyachur commented Jul 16, 2021

Details:

  • Use original ONNX instead of our fork

Tickets:

  • NA

@ilya-lavrenov
Copy link
Contributor

Could you please merge with latest master?

@ilya-lavrenov
Copy link
Contributor

@ilyachur please, rebase to latest master

@ilya-lavrenov
Copy link
Contributor

Looks like you also need to remove ONNX_BUILD_SHARED_LIBS definition when build onnx. Please, check it in the original changes

@ilyachur ilyachur marked this pull request as ready for review August 3, 2021 08:39
@ilyachur ilyachur requested review from a team, mbencer, postrational and tsocha August 3, 2021 08:39
@ilyachur ilyachur added this to the 2022.1 milestone Aug 3, 2021
@ilyachur ilyachur enabled auto-merge (squash) August 3, 2021 09:41
@ilya-lavrenov
Copy link
Contributor

Can we switch to latest v1.9.0?

@ilya-lavrenov ilya-lavrenov self-assigned this Aug 3, 2021
@ilyachur
Copy link
Contributor Author

ilyachur commented Aug 4, 2021

Can we switch to latest v1.9.0?

It will be a separate activity. Because we should make some additional changes in ONNX importer.

@ilyachur ilyachur merged commit 40a5ce2 into openvinotoolkit:master Aug 4, 2021
@ilyachur ilyachur deleted the original_onnx branch August 4, 2021 05:08
rnugmanx pushed a commit to rnugmanx/openvino that referenced this pull request Aug 26, 2021
* Merged ONNX editor and ONNX importer to the one folder

* Try to migrate to original ONNX

* Remove ONNX_BUILD_SHARED_LIBS

* Update ONNX
andrei-cv pushed a commit to andrei-cv/openvino that referenced this pull request Aug 30, 2021
* Merged ONNX editor and ONNX importer to the one folder

* Try to migrate to original ONNX

* Remove ONNX_BUILD_SHARED_LIBS

* Update ONNX
akuporos pushed a commit to akuporos/openvino that referenced this pull request Sep 29, 2021
* Merged ONNX editor and ONNX importer to the one folder

* Try to migrate to original ONNX

* Remove ONNX_BUILD_SHARED_LIBS

* Update ONNX
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: ONNX FE OpenVINO ONNX FrontEnd
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants